Skip to content

Update outdated bundle message - removed deprecation date #11230

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

manvkaur
Copy link
Contributor

@manvkaur manvkaur commented Aug 1, 2025

Issue describing the changes in this PR

Updated the outdated bundle warning message to be generic as the deprecation date is getting postponed.

resolves #10941

Pull request checklist

IMPORTANT: Currently, changes must be backported to the in-proc branch to be included in Core Tools and non-Flex deployments.

  • Backporting to the in-proc branch is not required
    • Otherwise: Link to backporting PR
  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes should not be added to the release notes for the next release
    • Otherwise: I've added my notes to release_notes.md
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • My changes do not require diagnostic events changes
    • Otherwise: I have added/updated all related diagnostic events and their documentation (Documentation issue linked to PR)
  • I have added all required tests (Unit tests, E2E tests)

Additional information

Additional PR information

@manvkaur manvkaur marked this pull request as ready for review August 13, 2025 20:54
@Copilot Copilot AI review requested due to automatic review settings August 13, 2025 20:54
@manvkaur manvkaur requested a review from a team as a code owner August 13, 2025 20:54
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the outdated extension bundle warning message to remove specific deprecation dates and make it more generic. The change simplifies the warning system by removing date-based logic that checked whether the deprecation date had passed.

  • Simplified the warning message to be generic without specific deprecation dates
  • Removed date-based conditional logic for displaying different warning messages
  • Updated corresponding test assertions to match the new warning format

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/WebJobs.Script/Diagnostics/Extensions/LoggerExtension.cs Removed date-based logic and consolidated warning messages into a single generic message
test/WebJobs.Script.Tests/Description/FunctionAppValidationServiceTests.cs Updated test assertions to check for the new generic warning message format
release_notes.md Added entry documenting the outdated bundle message update

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extension Bundles Retirement Messaging
2 participants